Skip to content

feat: #110/대시보드 수정 페이지 컴포넌트로 분리#109

Merged
codeit-kkm merged 1 commit intomainfrom
feat/#104/dashboard-edit-page
Mar 30, 2025
Merged

feat: #110/대시보드 수정 페이지 컴포넌트로 분리#109
codeit-kkm merged 1 commit intomainfrom
feat/#104/dashboard-edit-page

Conversation

@hyeonjiroh
Copy link
Owner

@hyeonjiroh hyeonjiroh commented Mar 30, 2025

#️⃣ Issue Number

#110


📝 요약

  • 대시보드 페이지를 대시보드 수정, 구성원, 초대 내역 컴포넌트로 분리

🛠️ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 변수명 변경 등)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

@hyeonjiroh hyeonjiroh self-assigned this Mar 30, 2025
@vercel
Copy link

vercel bot commented Mar 30, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
taskify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 30, 2025 2:24pm

@github-actions github-actions bot requested a review from jihye5081 March 30, 2025 14:24
@codeit-kkm codeit-kkm merged commit 70a3bd0 into main Mar 30, 2025
3 checks passed
@hyeonjiroh hyeonjiroh deleted the feat/#104/dashboard-edit-page branch March 30, 2025 14:27
@hyeonjiroh hyeonjiroh added refactor and removed feat labels Mar 30, 2025
@hyeonjiroh hyeonjiroh changed the title feat: #108/대시보드 수정 페이지 컴포넌트로 분리 feat: #110/대시보드 수정 페이지 컴포넌트로 분리 Mar 30, 2025
const router = useRouter();

return (
<div>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

혹시 div로 감싼 이유가 있을까요?

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

빠르게 분리하느라 필요 없는 div 태그를 지우는 걸 잊었네요 지우겠습니다!

>
<Image
src={BackIcon}
className="scale-x-[-1] w-[18px] h-[18px] tablet:w-[18px] tablet:h-[18px]"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아래처럼 한번에 적용 가능해요

w-[18px] h-[18px] -> size-[18px]

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 감사합니다! 가로세로 높이가 같은 경우에는 알려주신 방법으로 스타일을 적용하도록 하겠습니다!

className="scale-x-[-1] w-[18px] h-[18px] tablet:w-[18px] tablet:h-[18px]"
alt=""
/>
<div className="font-medium text-md text-gray-800 tablet:text-lg">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

돌아가기는 텍스트를 담당하는거같은데 p, span같은 태그를 활용해주세요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants